-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add design tokens to monorepo #7948
Conversation
Designer/manuelacosta
… into designer/jacqueskeet
First push after fix
Merge pull request #81 from Esri/designer/jacqueskeet
…/7180-1.0.0-design-tokens # Conflicts: # .release-please-manifest.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks for resubmitting this one, @alisonailea!
🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️
🕹️🪙🪙🪙🪙🪙🕹️🕹️🪙🪙🕹️🕹️🪙🕹️🕹️🪙🕹️🪙🪙🪙🪙🕹️🪙🕹️🕹️🕹️🪙🕹️🕹️🪙🪙🪙🕹️🪙🕹️
🕹️🕹️🕹️🪙🕹️🕹️🕹️🪙🕹️🕹️🪙🕹️🪙🕹️🪙🕹️🕹️🪙🕹️🕹️🕹️🕹️🪙🪙🕹️🕹️🪙🕹️🪙🕹️🕹️🕹️🕹️🪙🕹️
🕹️🕹️🕹️🪙🕹️🕹️🕹️🪙🕹️🕹️🪙🕹️🪙🪙🕹️🕹️🕹️🪙🪙🪙🕹️🕹️🪙🕹️🪙🕹️🪙🕹️🕹️🪙🪙🕹️🕹️🪙🕹️
🕹️🕹️🕹️🪙🕹️🕹️🕹️🪙🕹️🕹️🪙🕹️🪙🕹️🪙🕹️🕹️🪙🕹️🕹️🕹️🕹️🪙🕹️🕹️🪙🪙🕹️🕹️🕹️🕹️🪙🕹️🕹️🕹️
🕹️🕹️🕹️🪙🕹️🕹️🕹️🕹️🪙🪙🕹️🕹️🪙🕹️🕹️🪙🕹️🪙🪙🪙🪙🕹️🪙🕹️🕹️🕹️🪙🕹️🪙🪙🪙🕹️🕹️🪙🕹️
🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️🕹️
Once this lands, let's make sure to:
- follow-up on comments brought up in feat: calcite-design-tokens in monorepo package #7621
- migrate applicable issues from https://github.com/Esri/calcite-design-tokens/
- archive https://github.com/Esri/calcite-design-tokens/ with a respective README update when ready
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple small cleanups but otherwise LGTM! 🚀
"name": "@esri/calcite-design-tokens", | ||
"version": "1.0.0", | ||
"description": "Esri's Calcite Design System Tokens", | ||
"homepage": "https://github.com/Esri/calcite-design-tokens", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The URLs in this package.json need to be updated again
@@ -0,0 +1,177 @@ | |||
{ | |||
"[comp-name]": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file can be deleted right?
@@ -0,0 +1,238 @@ | |||
{ | |||
"accordion": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This _backup file can be deleted right?
remove missing JS files (will be in v2) point repo and bugs urls to calcite-design-system
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thanks for working through all our review items, I know it was a lot! 🏆
* origin/main: (83 commits) chore: add design tokens to monorepo (#7948) test(input-time-zone): restore common tests (#7963) chore: release next fix(input-time-zone): fix error caused by time zone group filtering (#7971) chore: release next fix(table): Improve scrollbar display (#7967) build: bump timezone-groups (#7962) chore: release next fix(flow-item): update collapsed property when collapse button is clicked (#7960) chore: release main (#7945) chore: release next fix(input-time-zone): fix handling of unknown and cityless time zones from offset display mode (#7947) fix(combobox): fix issue causing value to be cleared when selecting an item (Windows + trackpad) (#7954) chore(deps): update calcite-ui-icons to 3.24.7 (#7949) test(flow-item, panel): add tests for internal property. (#7931) chore: release next fix(panel): fix collapse action title and reverse icon direction (#7927) chore: update issue template labels to support workflows (#7938) build(deps): bump timezone-groups (#7933) build(deps): update dependency @esri/calcite-ui-icons to v3.24.6 (#7925) ...
Related Issue: #7180
Summary
Merge https://github.com/Esri/calcite-design-tokens/tree/merge/1.0.0-calcite-design-system with https://github.com/Esri/calcite-design-system/tree/main
Includes
calcite-design-tokens as of release 1.0.0
updates to design tokens (removing temp files and backups)
update to github actions to build tokens for visual regression tests